-
-
Notifications
You must be signed in to change notification settings - Fork 244
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[14.0][RFC] payment_pagseguro: refactor charge to orders endpoint #2309
base: 14.0
Are you sure you want to change the base?
[14.0][RFC] payment_pagseguro: refactor charge to orders endpoint #2309
Conversation
f9324b3
to
612d4d7
Compare
vale a pena vc tb checkar os últimos commits na V12, porque até onde eu lembro nem tudo tinha sido portado para a V14 neste módulo... |
Fato |
"card": { | ||
"encrypted": self.payment_token_id.pagseguro_card_token, | ||
}, | ||
"items": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eu colocaria algo assim para listar as sale_order_line:
items = []
for line in self.sale_order_ids[0].order_line:
item = {
"reference_id": line.product_id.default_code,
"name": line.product_id.name,
"quantity": line.product_uom_qty,
"unit_amount": int(line.price_unit * 100),
}
items.append(item)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@augustodinizl se possível, pega a branch desta PR e submete uma PR para o fork da https://github.com/escodoo/l10n-brazil fazendo favor.
Na situação atual, é um pedido enviado para o pagseguro é o modelo de pre-autorização sem captura de cobrança de automática. Para usar o modelo de pré-autorização + captura, deve ser definido o parametro 'capture' |
612d4d7
to
f4f7579
Compare
f4f7579
to
29bbfc8
Compare
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
Bom dia @marcelsavegnago, Tem previsão de dar continuidade nesse PR? Chegou a verificar os testes? |
Grande @mileo estou usando esta PR em um cliente mas não consegui dar andamento nela. Se tiver alguem que pode dar uma força.. eu agradeço. |
Foi iniciado o desenvolvimento dessa feature aqui: https://github.com/kmee/l10n-brazil/tree/14.0-pagseguro-pix-boleto Pode ser que te ajude a seguir com esse desenvolvimento. |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
@mileo nesta PR eu fiz alguns ajustes para conseguir fazer a comunicação com o pagseguro. Porém, se tiver interesse fique a vontade para trabalhar na refatoração/implementação necessária para adpatar as novas necessidades do pagseguro.
Testei o envio e a captura.. ainda falta testar o cancelamento.
obs. alguns dados estão chumbados no código